Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show only records after June 20, 2016 #22

Merged
merged 1 commit into from
Jul 27, 2016
Merged

Conversation

MusikAnimal
Copy link
Member

Bug: T138317

@@ -113,6 +113,9 @@ public function getPlagiarismRecords( $n = 50, $options ) {
$filters[] = "wp_project IN ($bindParams)";
}

// show only records after June 20, 2016; See phab:T138317
$filters[] = "diff_timestamp > '20160620000000'";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems strange to put quotation marks around the number here.

@kaldari
Copy link
Collaborator

kaldari commented Jul 27, 2016

+2

@MusikAnimal MusikAnimal merged commit 077e411 into master Jul 27, 2016
@MusikAnimal MusikAnimal deleted the no-old-records branch July 27, 2016 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants